Skip to content

Conversation

@geminorum
Copy link
Contributor

@geminorum geminorum commented Dec 14, 2025

to address the following:

  • the generated styles is not loading on rtl. entire file must be repalced via wp_style_add_data().
  • the formation of scheme prefix + domain input must be preserved on rtl.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request implements RTL (Right-to-Left) language support enhancements for the WP Multi Network plugin. The changes ensure that RTL stylesheets are properly loaded as complete replacements rather than as overrides, and improve the visual presentation of the domain input field by wrapping the scheme prefix and domain input together to maintain their connection in RTL layouts.

  • Configured WordPress to load complete RTL stylesheet replacements using wp_style_add_data() with 'replace' mode
  • Restructured the domain input HTML to use a wrapper span and HTML comments to eliminate unwanted whitespace between the scheme prefix and input field
  • Added "code" class styling to domain and path inputs for monospace font rendering

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
wp-multi-network/includes/classes/class-wp-ms-networks-admin.php Adds RTL stylesheet support by registering 'rtl' => 'replace' mode and handling minified suffix variations
wp-multi-network/includes/metaboxes/edit-network.php Wraps scheme prefix and domain input in a code-styled container, uses HTML comments to prevent whitespace, and applies monospace styling to technical inputs

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@geminorum geminorum requested a review from JJJ December 18, 2025 03:07
@JJJ JJJ merged commit cdaac05 into stuttter:master Dec 18, 2025
19 checks passed
@geminorum geminorum deleted the rtl-enhancements branch December 18, 2025 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants